Skip to content

test: add 50% coverage threshold ratchet#227

Open
brendanjryan wants to merge 1 commit intomainfrom
test/coverage-threshold
Open

test: add 50% coverage threshold ratchet#227
brendanjryan wants to merge 1 commit intomainfrom
test/coverage-threshold

Conversation

@brendanjryan
Copy link
Collaborator

@brendanjryan brendanjryan commented Mar 24, 2026

Summary

Add coverage thresholds to vitest.config.ts so CI fails on coverage regressions.

Current Thresholds

Category Floor Current
Statements 50% 44.6%
Branches 50% 41.8%
Functions 50% 54.3% ✅

--

This will fail until we land more coverage in key areas

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 24, 2026

Open in StackBlitz

npm i https://pkg.pr.new/mppx@227

commit: dc8a7b1

Set coverage floor at 50% for statements, branches, and functions.
CI will fail if coverage drops below these thresholds.

Current measured coverage (without localnet):
- Statements: 44.6% (needs ~5.4% lift)
- Branches: 41.8% (needs ~8.2% lift)
- Functions: 54.3% (already passing)
@brendanjryan brendanjryan force-pushed the test/coverage-threshold branch from 328970d to dc8a7b1 Compare March 24, 2026 18:38
@brendanjryan
Copy link
Collaborator Author

N.b. this should fail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant